Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

feat(ios): add time Profiler #188

Merged
merged 21 commits into from
Jan 19, 2024
Merged

feat(ios): add time Profiler #188

merged 21 commits into from
Jan 19, 2024

Conversation

GuillaumeEgret
Copy link
Contributor

No description provided.

@netlify Netlify
Copy link

netlify bot commented Jan 15, 2024 •
edited
Loading

✅ Deploy Preview for flashlight-docs canceled.

Name Link
🔨 Latest commit 6479ccd
🔍 Latest deploy log https://app.netlify.com/sites/flashlight-docs/deploys/65aa80258a651400089bd097

@GuillaumeEgret GuillaumeEgret force-pushed the prepare-ios branch 2 times, most recently from 6cf5e6d to 36deab8 Compare January 15, 2024 13:42
@github-actions GitHub Actions
Copy link

github-actions bot commented Jan 15, 2024 •
edited
Loading

Flashlight iOS Report 🔦

@flashlight-bot
Copy link
Contributor

Flashlight Automated Report 🔦

@flashlight-bot
Copy link
Contributor

Flashlight Automated Report 🔦

Almouro
Almouro reviewed Jan 16, 2024
View reviewed changes
@flashlight-bot
Copy link
Contributor

Flashlight Automated Report 🔦

Almouro
Almouro reviewed Jan 17, 2024
View reviewed changes
@flashlight-bot
Copy link
Contributor

Flashlight Automated Report 🔦

@Almouro
Copy link
Member

Almouro commented Jan 17, 2024

Awesome, only thing missing is to do something about the "high cpu thread usage" since this method doesn't support thread by thread

image

@GuillaumeEgret
feat: add ios profiler
fc3308b
@GuillaumeEgret
feat: create a device manager for ios profiler
69c1bde
@GuillaumeEgret
feat: kill the ios app before test
3ff9d9e
@GuillaumeEgret
feat(test): allow test command for ios profiler
32f2960
@GuillaumeEgret
fix CI
088a412
@GuillaumeEgret
feat: add readme
85092ff
@GuillaumeEgret
fix(test): prevent breaking change for android users
ee04594
@GuillaumeEgret
chore(readme): add dependencies in readme
135ca12
@GuillaumeEgret
chore(ios profiler): replace default values by null
50118ed
@GuillaumeEgret
feat(ios profiler): poll data more often
6aad828
@GuillaumeEgret
fix(web-reporter): report crashed if series was empty
763c3d6
@GuillaumeEgret
feat(CPUReport): hide Thread graph if there is only one thread
41af1d5
@GuillaumeEgret
chore(ios profiler): set lastFps and lastCpu to null
a66e167
@GuillaumeEgret
feat(profiler): add stop app function in profiler
9eb7423
@GuillaumeEgret
chore(ios profiler): set same polling interval than android
5462e58
@GuillaumeEgret
feat(ReportSummaryCard): hide high cpu usage if there is no thread pe…
912252c 
…r thread details
@GuillaumeEgret
feat(readme): add know limits in readme
74e98ff
@flashlight-bot
Copy link
Contributor

Flashlight Automated Report 🔦

@flashlight-bot
Copy link
Contributor

Flashlight Automated Report 🔦

@flashlight-bot
Copy link
Contributor

Flashlight Automated Report 🔦

@Almouro Almouro changed the title Add iOS Profiler feat(ios): add time Profiler Jan 19, 2024
@Almouro Almouro merged commit b04984d into main Jan 19, 2024
7 checks passed
@Almouro Almouro deleted the prepare-ios branch January 19, 2024 16:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Almouro Almouro Almouro approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@GuillaumeEgret @flashlight-bot @Almouro